Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes for Recent Google requirements changes #6

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

muratiakos
Copy link

@muratiakos muratiakos commented Nov 26, 2023

This PR updates the key dependencies to meet recent requirement changes from Google Play store to accept new applications based on https://developer.android.com/google/play/requirements/target-sdk and also fixes some broken links. The changes are namely:

  • Bump SDK version to 34
  • Upgrade Gradle to 8.7
  • Bump Kotlin version and other dependencies to supported to fix compatibility issues
  • Fix broken Volvo Automotive page link

The changes were tested successfully on the official Volvo XC40 VD:
image

Other possible updates and refactoring were intentionally not been made to keep the change-set small and focused on original functionality.

[email protected] , @AshWave - Thanks for this code and considering my changes as part of your your review.

@AshWave
Copy link
Collaborator

AshWave commented Oct 11, 2024

@muratiakos Thanks for updating this! Apologies for looking at this after a.... while! Changes look reasonable, so I'm happy to approve it, but I think it needs to be updated yet again, since it hasn't been maintained for a while :)

@muratiakos
Copy link
Author

@muratiakos Thanks for updating this! Apologies for looking at this after a.... while! Changes look reasonable, so I'm happy to approve it, but I think it needs to be updated yet again, since it hasn't been maintained for a while :)

Thanks @AshWave ! I have updated the PR and bumped the versions and dependecies too. Feel free to test and merge if you are happy with these!

@muratiakos muratiakos requested a review from AshWave November 14, 2024 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants